fix(tui): draw the progress bar from apk's first package - #41
Conversation
apk right-pads the done counter to the total's width, so early lines read ( 1/263). apkCountRe required a digit right after (, so it matched no line until the counter reached the total's digit count. On a 263-package apply the bar stayed blank for the first 99 packages. \s* absorbs the padding, matching what pacmanCountRe already does.
|
Warning Review limit reached
Next review available in: 28 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
1 participant
The install/apply progress bar stayed blank for the whole early part of a big install.
apk right-pads the done counter to the total's width, so early lines read
( 1/263).apkCountRerequired a digit immediately after(, so it matched no line until the counter reached the total's digit count. On the 263-package xfce apply that meant no bar for the first 99 packages; the panel showed only the step and last log line.pacmanCountRealready tolerates the padding with\s*; apk's regex now does too.Also helps the install phase, since setup-alpine's apk output pads the same way.
Test: added a padded-counter case to TestParseProgress.